-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add missing config settings to type declarations in dist/htmx.d.ts #1936
Conversation
Add missing config settings to type declarations in dist/htmx.d.ts
This is a follow-up to earlier PR, closed due to my git incompetence: #1934 |
thank you! |
Shoot, I didn't realize your actual source of truth for type declarations was in the This was pulled against dist, not src. Sorry about that @alexpetros, @1cg Am I correct it should have been against this file instead? https://github.com/bigskysoftware/htmx/blob/dev/src/htmx.d.ts Also, slightly off topic, are you open to a PR adding the various hx attributes as a new export in the |
Yes it should have been, that's my bad for not catching that.
What's the point of typing HTML attributes, what does that have to do with typescript? (I'm sure there's an answer, I just don't know it) |
For auto complete. So when you are say here, mid-typing:
You can hit ctrl-space or whatever your editor's key combo for autocomplete is, and see all the possible htmx attributes. It's great if your brain is old and forgetful like me. And some take booleans or strings -- so you'd get a type error if you pass a boolean to an attribute that is expecting a string. |
Yup, we're more than open to PRs that update these. |
Add missing config settings to type declarations in dist/htmx.d.ts